Update and rename docs/fileBytes.md to docs/developers/fileBytes.md#95
Update and rename docs/fileBytes.md to docs/developers/fileBytes.md#95DeveloperTryingToCodeLikeOtherOfThem wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the header of the file bytes documentation and moves the file to a new subdirectory. Feedback highlights that moving the file breaks the documentation index generation script, which currently lacks support for recursive directory traversal and correct URL construction for subdirectories.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e6391e3754
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -1,4 +1,4 @@ | |||
| ## How Many Bytes Does a File Need? | |||
| ## Bytes for a File | |||
There was a problem hiding this comment.
Keep npm
files list in sync with renamed doc path
After moving this document to docs/developers/fileBytes.md, the package manifest still lists the old path (package.json files entry docs/fileBytes.md), so the renamed doc is no longer shipped in published tarballs. I verified with npm pack --dry-run in this repo: docs/developers/fileBytes.md is absent from the output. Any consumer expecting this doc in the package will now miss it until the manifest is updated.
Useful? React with 👍 / 👎.
No description provided.